Skip to content

fix: remove all remaining sudo, add SSH key auth for Incus#16

Merged
ethannortharc merged 1 commit intomainfrom
fix/remove-all-sudo-and-ssh-auth
Mar 11, 2026
Merged

fix: remove all remaining sudo, add SSH key auth for Incus#16
ethannortharc merged 1 commit intomainfrom
fix/remove-all-sudo-and-ssh-auth

Conversation

@ethannortharc
Copy link
Copy Markdown
Owner

Summary

  • Remove all remaining sudo from overlay.rs, nix/mod.rs, nix/rebuild.rs
  • Use bash -lc for commands needing NixOS PATH (find, mount, nix, etc.)
  • Fix devbox code on Incus:
    • Detect actual VM username instead of hardcoded "devbox"
    • Auto-inject host SSH pubkey into VM for passwordless auth
    • Ensure sshd is running in the VM
    • Include IdentityFile in SSH config

Test plan

  • devbox code hive-dev opens VS Code without password prompt
  • Overlay refresh works without errors
  • devbox layer status works on Incus

🤖 Generated with Claude Code

- Remove sudo from overlay.rs, nix/mod.rs, nix/rebuild.rs (same PAM
  issue as provision.rs — incus exec already runs as root)
- Use bash -lc for commands needing NixOS PATH (find, mount, nix, etc.)
- Fix devbox code on Incus: detect actual username instead of hardcoded
  "devbox", auto-inject SSH pubkey into VM for passwordless auth,
  ensure sshd is running

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ethannortharc ethannortharc merged commit 278d350 into main Mar 11, 2026
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant